Skip to content

Conversation

ChayimFriedman2
Copy link
Contributor

No description provided.

It didn't do anything (behaved like `item`), as with `enforceGranularity = false` (which is the default), the style of the current file is always preferred, regardless of the setting.

We could make it fail when the setting is `preserve` and the current file's style could not be detected, but that makes little sense.

It is a bit weird that the default is `crate` but `preserve` falls back to `item`, however that was the previous behavior.
Previously it was wrong for some combinations.
@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Oct 5, 2025
Copy link
Member

@ShoyuVanilla ShoyuVanilla left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I was going to ask about the intentions behind these changes, but the commit messages already explain them clearly 😅 Looks good to me

@ShoyuVanilla ShoyuVanilla added this pull request to the merge queue Oct 10, 2025
Merged via the queue into rust-lang:master with commit 87ccc2b Oct 10, 2025
16 checks passed
@rustbot rustbot removed the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Oct 10, 2025
@ChayimFriedman2 ChayimFriedman2 deleted the fix-insert-use branch October 10, 2025 08:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants